home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 2286 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  1.1 KB
  1. Path: ix.netcom.com!netnews
  2. From: miker3@ix.netcom.com (Mike Rubenstein)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Newbie question:  Is this code OK?
  5. Date: Fri, 19 Jan 1996 22:11:28 GMT
  6. Organization: Netcom
  7. Message-ID: <31000875.75333568@nntp.ix.netcom.com>
  8. References: <4dmebk$foq@pegasus.interpac.net> <4doajv$ov0$2@mhafn.production.compuserve.com>
  9. NNTP-Posting-Host: ix-dc6-09.ix.netcom.com
  10. X-NETCOM-Date: Fri Jan 19  2:11:27 PM PST 1996
  11. X-Newsreader: Forte Agent .99c/16.141
  12.  
  13. Bill Symmes <71046.623@CompuServe.COM> wrote:
  14.  
  15. > Why not just
  16. > 
  17. >     while (*string)
  18. >            if (isspace(*string++)
  19. >               numwords++;
  20.  
  21. How    many words in       this sentence?
  22.  
  23. Also, there's another problem in your code and others.  Assuming
  24. string is a pointer to char or array of char, isspace(*string++) may
  25. be invalid.  isspace() requires an int argument that is equal to a
  26. value valid for an unsigned char or to EOF.  If char is signed,
  27. *string may be a negative value that is invalid for isspace().  The
  28. test for space should be isspace((unsigned char) *string++).
  29.  
  30.  
  31. Michael M Rubenstein
  32.